Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add multiple assignment support to java.util.Collection's Fixes #459 #460

Merged
merged 1 commit into from Mar 22, 2013

Conversation

atambo
Copy link
Member

@atambo atambo commented Dec 22, 2012

No description provided.

@nirvdrum
Copy link
Contributor

Should this only work with java.util.List implementations? java.util.Set inherits from java.util.Collection, but Ruby doesn't exhibit the same multiple assignment behaviour with Sets, presumably because there is no ordering.

@atambo
Copy link
Member Author

atambo commented Dec 22, 2012

@nirvdrum, what about a Queue where there is ordering but it is not a List?

@atambo
Copy link
Member Author

atambo commented Dec 22, 2012

We'd also miss the SortedSet interface if we went List only. I feel like we should just do it for all collections since it will still work it just wont have any guarantees about order.

@nirvdrum
Copy link
Contributor

Ahh, more corner cases. I guess I don't have a strong issue with it being on java.util.Collection, other than it doesn't have 1:1 parity with Ruby's Set. Since #459 seemed to be about deviating behaviour, I figured I'd mention that this breaks in a different (probably less important) way.

headius added a commit that referenced this pull request Mar 22, 2013
add multiple assignment support to java.util.Collection's Fixes #459
@headius headius merged commit ff74833 into jruby:master Mar 22, 2013
@headius
Copy link
Member

headius commented Mar 22, 2013

Good fix, I like it. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants